Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update sumo builder utility #1906

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update sumo builder utility #1906

wants to merge 1 commit into from

Conversation

Gamenot
Copy link
Collaborator

@Gamenot Gamenot commented Mar 14, 2023 •
edited
Loading

This is a step toward resolving #619. This updates the sumo source builder to generalize for other Ubuntu versions. This will be used to simplify building(and rebuilding) the debug version of SUMO.

The end intention is the following:

$ sumo_builder.sh -u 20.04 -s v1_16_0 --no-persist
# -s, --sumo-tag will cause a checkout (needed if using remote)
# -r, --repository will use a specific repository (default https://github.com/eclipse/sumo)
$ sumo_builder.sh -u 20.04 -r ~/repos/sumo --debug
Builder does not exist, generating a new container...
Build triggered.
...
Build output to "./build"
$ sumo_builder.sh -u 20.04 -r ~/repos/sumo --debug
Builder exists, reusing the container...
Rebuild triggered.
...
Build output to "./build"

TODO

  • Target SUMO repository
  • Add debug build option
  • Option to persist builder container (default persist)
  • Consider python version option

@Gamenot
Copy link
Collaborator Author

Gamenot commented Apr 6, 2023

This exposed a funny bug in the CI where pytype test will run without any python files because the CI looks for .py which is in "README.pypi.md".

Not sure it is worth fixing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Gamenot